Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix: No document types allowed at root message #2338

Closed

Conversation

loivsen
Copy link
Contributor

@loivsen loivsen commented Sep 25, 2024

Description

When trying to create a new document type under content (at the root), and there is no allowed document types available, show the correct message and remove the edit button.
The button has been removed as there are multiple options when creating a document type (it was not there in v13 either)

Fixes umbraco/Umbraco-CMS#17107

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (minor updates related to the tooling or maintenance of the repository, does not impact compiled assets)

nielslyngsoe and others added 30 commits September 12, 2024 22:23
…dCondition

Feature: Adds new method to extensionRegistry for addCondition & addConditions
…-credential-id

Add client credential id pattern validation
… condition loads first, as order of conditions are important
…y wait until the manifest is registered so late extensions will work as well

Seems to work with my hack in an entrypoint and the console logs the extension with the updated condition but its the tests I am struggling to get to work.

As this code is heavily worked from me trying to Google, read RXJS docs and talking with Copilot I am not fully in confident in this approach
… but test says otherwise compared to using it in entrypoint
…ddCondtions

As the order of conditions is not important, as all conditions set on an extensions must be true/valid
Still need to uncomment & reinvestigate the late loading test
nielslyngsoe and others added 26 commits September 23, 2024 15:06
…tional-options

Feature: Entity Actions additional options
…al-validation

Implement validation in 'insert field' modal
…ttings-token

Fix: move user permission settings token
…-picker-modal

Fix: Move property editor UI picker modal
Bumps [rollup](https://github.com/rollup/rollup) from 4.21.0 to 4.22.4.
- [Release notes](https://github.com/rollup/rollup/releases)
- [Changelog](https://github.com/rollup/rollup/blob/master/CHANGELOG.md)
- [Commits](rollup/rollup@v4.21.0...v4.22.4)

---
updated-dependencies:
- dependency-name: rollup
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
@loivsen loivsen changed the base branch from main to release/14.3 September 25, 2024 11:46
@loivsen loivsen closed this Sep 25, 2024
@loivsen loivsen deleted the v15/bugfix/no-document-types-allowed-at-root-message branch September 25, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Edit permissions for this Document Type" gives APIError
6 participants